Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for NVM_SILENT environment variable #91 #102

Closed
wants to merge 1 commit into from

Conversation

brianmgray
Copy link

@brianmgray brianmgray commented Mar 22, 2024

I ran into the same problem discussed in this PR: #91

Since the PR was nearly complete, I just addressed the comments and submitted it here.

I have tested it and indeed I no longer get the warning (and continue to switch with no output):

BEFORE
Screenshot 2024-03-22 at 4 14 45 PM

AFTER
Screenshot 2024-03-22 at 4 24 06 PM

@brianmgray
Copy link
Author

@gentslava are you able to review?

@gentslava
Copy link

@brianmgray I'm not able to review. Don't have such rights for this repo.
You can contribute to my fork and use it instead.
It seems this repository is no longer maintained.

@brianmgray
Copy link
Author

I saw you merged my PR - I switched over to your fork, and it's working great. Thanks!

@brianmgray brianmgray closed this Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants